Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lets observer with SecHUD enabled see sec records on examine. #4890

Merged
merged 2 commits into from
Nov 13, 2023

Conversation

Birdtalon
Copy link
Contributor

@Birdtalon Birdtalon commented Nov 10, 2023

About the pull request

Lets observer with Security HUD switched on see and read security records.

Explain why it's good for the game

Entertaining to see why the Bravo SL is wanted on the frontline after you're dead.

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑
add: Observer with Security HUD enabled can examine and read security records.
/:cl:

@github-actions github-actions bot added the Feature Feature coder badge label Nov 10, 2023
Copy link
Member

@harryob harryob left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks fine otherwise

code/modules/mob/living/carbon/human/examine.dm Outdated Show resolved Hide resolved
@Birdtalon
Copy link
Contributor Author

looks fine otherwise

Fastest reviewer in the west. Changes made.

Copy link
Member

@harryob harryob left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

seems fun

@harryob harryob added this pull request to the merge queue Nov 13, 2023
Merged via the queue into cmss13-devs:master with commit 45cbeba Nov 13, 2023
26 checks passed
cm13-github added a commit that referenced this pull request Nov 13, 2023
github-merge-queue bot pushed a commit that referenced this pull request Nov 16, 2023
# About the pull request

Adds missing newline after [View] so that flavour text displays below.

# Explain why it's good for the game
# Testing Photographs and Procedure
<details>
<summary>Screenshots & Videos</summary>

Put screenshots and videos here with an empty line between the
screenshots and the `<details>` tags.

</details>


# Changelog
:cl:
fix: Adds missing newline when viewing sec records as observer.
/:cl:

---------

Co-authored-by: forest2001 <[email protected]>
@Birdtalon Birdtalon deleted the observesecrecords branch February 25, 2024 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Feature coder badge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants